Skip to content

Conversation

Copy link

Copilot AI commented Aug 7, 2025

This PR upgrades @api-ts/typed-express-router from version ^1.1.13 to 2.0.0 to take advantage of the latest features and improvements in the typed Express router library.

Changes

  • Updated @api-ts/typed-express-router dependency from ^1.1.13 to ^2.0.0 in package.json
  • Generated package-lock.json with all updated dependencies
  • Verified TypeScript compilation compatibility with the new version

Analysis

After analyzing the codebase, I found that:

  1. 5 files use createRouter from @api-ts/typed-express-router:

    • src/advancedWalletManager/routers/advancedWalletManagerApiSpec.ts
    • src/advancedWalletManager/routers/healthCheck.ts
    • src/api/master/routers/awmExpressHealth.ts
    • src/api/master/routers/healthCheck.ts
    • src/api/master/routers/masterBitGoExpressApiSpec.ts
  2. All required exports remain available: createRouter, TypedRequestHandler, and WrappedRouter are all present in v2.0.0.

The upgrade is straightforward since we only use the basic router creation functionality and don't rely on any of the deprecated error handling patterns.

Fixes #108.
Ticket: DX-1672


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copy link
Author

Copilot AI commented Aug 7, 2025

@starfy84 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Copilot AI changed the title [WIP] Bump typed-express-router to 2.0.0 feat: bump typed-express-router to 2.0.0 Aug 7, 2025
Copilot AI requested a review from starfy84 August 7, 2025 16:53
@starfy84
Copy link
Contributor

@copilot redo this, we just need to update package.json with the new version, update package-lock.json with the updated package, and potentially modify code

Copy link
Author

Copilot AI commented Aug 11, 2025

@copilot redo this, we just need to update package.json with the new version, update package-lock.json with the updated package, and potentially modify code

Done! I've updated the approach to focus only on the essential changes:

  1. ✅ Updated package.json with the new version (^2.0.0)
  2. ✅ Generated package-lock.json with the updated package and all dependencies
  3. ✅ Verified no code changes are needed - the upgrade is backwards compatible for our usage
  4. ✅ Removed unnecessary package files from the previous approach

The package upgrade is complete and working correctly. See commit 2c31359.

@starfy84 starfy84 marked this pull request as ready for review August 11, 2025 14:03
@starfy84 starfy84 requested review from a team, adityasoni-bitgo and satyamkotikalapudi and removed request for a team August 20, 2025 14:26
@kaustubhbitgo
Copy link

Flushing reviews!

@zahin-mohammad zahin-mohammad merged commit 6f6d7a3 into master Aug 22, 2025
4 checks passed
@zahin-mohammad zahin-mohammad deleted the copilot/fix-108 branch August 22, 2025 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bump typed-express-router to 2.0.0

4 participants